Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[VT]: Add a way to track VT float attributes #482

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

ad3154
Copy link
Member

@ad3154 ad3154 commented Jul 8, 2024

Describe your changes

This lets you use the VT state helper to deal with float attributes. Previously, float values were probably not handled correctly.
I was seeing that there were warnings in the seeder example about this:
image

How has this been tested?

  • Run Seeder Example and test variable scale factors

@ad3154 ad3154 added enhancement New feature or request iso: virtual terminal Related to the ISO-11783:7 standard labels Jul 8, 2024
@ad3154 ad3154 requested a review from GwnDaan July 8, 2024 23:58
@ad3154 ad3154 self-assigned this Jul 8, 2024
@ad3154 ad3154 force-pushed the adrian/vt-float-attributes branch 2 times, most recently from 38a6923 to 4c83485 Compare July 9, 2024 00:18
Copy link

sonarqubecloud bot commented Jul 9, 2024

ad3154 and others added 3 commits January 8, 2025 12:44
This lets you use the VT state helper to deal with float attributes.
Previously, float values may not have been handled correctly.
…eed messages

and fixed incorrect formatting specifiers in `isobus_virtual_terminal_client_update_helper.cpp`
@GwnDaan GwnDaan force-pushed the adrian/vt-float-attributes branch from edb3675 to cd0961d Compare January 8, 2025 12:12
Copy link

sonarqubecloud bot commented Jan 8, 2025

@GwnDaan GwnDaan merged commit bbac440 into main Jan 8, 2025
11 checks passed
@GwnDaan GwnDaan deleted the adrian/vt-float-attributes branch January 8, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request iso: virtual terminal Related to the ISO-11783:7 standard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants